Skip to content

[codex] Update verifier Codex CLI pin - #1901

Merged
stranske merged 1 commit into
mainfrom
codex/verifier-codex-cli-0125
Apr 26, 2026
Merged

[codex] Update verifier Codex CLI pin#1901
stranske merged 1 commit into
mainfrom
codex/verifier-codex-cli-0125

Conversation

@stranske

@stranske stranske commented Apr 26, 2026

Copy link
Copy Markdown
Owner

Source: Issue #1836

Related to campaign issue #1836

Automated Status Summary

Scope

Sync/Dependabot Campaign Queue

Remote GitHub Actions owns discovery for sync-generated and Dependabot PR rounds. Local Codex should only claim items from this issue when needs-local-codex work is queued.

Summary

Context for Agent

Related Issues/PRs

Tasks

  • Updated: 2026-04-26T13:55:31.783Z
  • Repos checked: 11/11
  • Open sync PRs: 461
  • Open Dependabot PRs: 0
  • Active review threads queued: 710
  • Items needing local Codex: 0
  • Actionable local Codex items: 0
  • Claimable local Codex items: 0
  • Source-fixed candidates: 0
  • Superseded sync candidates: 120
  • Finished local results without published source changes: 0
  • Claimed local Codex items: 0
  • Next claim lease expires: -

Acceptance criteria

  • Status: local-codex-superseded-sync-candidate
  • Source repo: stranske/Workflows
  • Source sync state: superseded (PR a3e3f6fa23d5 / current ede2607c77be)
    • scripts/aggregate_agent_metrics.py:274 (copilot-pull-request-reviewer): The unsupported-model list is configured via UNSUPPORTED_VERIFIER_MODELS here, but the terminal-disposition coverage ...

Head SHA: 173c8a8
Latest Runs: ❔ in progress — Gate
Required: gate: ❔ in progress

Workflow / Job Result Logs
Gate ❔ in progress View run
Health 40 Sweep ✅ success View run
Health 44 Gate Branch Protection ❔ in progress View run
Health 45 Agents Guard ✅ success View run
Health 50 Security Scan ❔ in progress View run
Health 73 Template Completeness ✅ success View run
Maint 52 Validate Workflows ✅ success View run
PR 11 - Minimal invariant CI ✅ success View run
Selftest CI ✅ success View run
Validate Sync Manifest ✅ success View run

@stranske-keepalive

stranske-keepalive Bot commented Apr 26, 2026

Copy link
Copy Markdown
Contributor

Automated Status Summary

Head SHA: 4e32c8b
Latest Runs: ⏳ pending — Gate
Required contexts: Gate / gate, Health 45 Agents Guard / guard
Required: core tests (3.12): ⏳ pending, core tests (3.13): ⏳ pending, docker smoke: ⏳ pending, gate: ⏳ pending

Workflow / Job Result Logs
(no jobs reported) ⏳ pending

Coverage Overview

  • Coverage history entries: 1

Coverage Trend

Metric Value
Current 93.12%
Baseline 85.00%
Delta +8.12%
Minimum 70.00%
Status ✅ Pass

Top Coverage Hotspots (lowest coverage)

File Coverage Missing
src/cli_parser.py 81.8% 4
src/percentile_calculator.py 95.0% 1
src/aggregator.py 95.0% 2
src/__init__.py 100.0% 0
src/ndjson_parser.py 100.0% 0

Updated automatically; will refresh on subsequent CI/Docker completions.


Keepalive checklist

Scope

Sync/Dependabot Campaign Queue

Remote GitHub Actions owns discovery for sync-generated and Dependabot PR rounds. Local Codex should only claim items from this issue when needs-local-codex work is queued.

Summary

Context for Agent

Related Issues/PRs

Tasks

  • Updated: 2026-04-26T13:55:31.783Z
  • Repos checked: 11/11
  • Open sync PRs: 461
  • Open Dependabot PRs: 0
  • Active review threads queued: 710
  • Items needing local Codex: 0
  • Actionable local Codex items: 0
  • Claimable local Codex items: 0
  • Source-fixed candidates: 0
  • Superseded sync candidates: 120
  • Finished local results without published source changes: 0
  • Claimed local Codex items: 0
  • Next claim lease expires: -

Acceptance criteria

  • Status: local-codex-superseded-sync-candidate
  • Source repo: stranske/Workflows
  • Source sync state: superseded (PR a3e3f6fa23d5 / current ede2607c77be)
    • scripts/aggregate_agent_metrics.py:274 (copilot-pull-request-reviewer): The unsupported-model list is configured via UNSUPPORTED_VERIFIER_MODELS here, but the terminal-disposition coverage ...

@stranske-keepalive

stranske-keepalive Bot commented Apr 26, 2026

Copy link
Copy Markdown
Contributor

🤖 Keepalive Loop Status

PR #1901 | Agent: Codex | Iteration 0/5

Current State

Metric Value
Iteration progress [----------] 0/5
Action wait (missing-agent-label)
Disposition skipped (transient)
Gate success
Tasks 0/4 complete
Timeout 45 min (default)
Timeout usage 4m elapsed (10%, 41m remaining)
Keepalive ❌ disabled
Autofix ❌ disabled

🔍 Failure Classification

| Error type | infrastructure |
| Error category | resource |
| Suggested recovery | Confirm the referenced resource exists (repo, PR, branch, workflow, or file). |

@stranske-keepalive

stranske-keepalive Bot commented Apr 26, 2026

Copy link
Copy Markdown
Contributor
Keepalive Work Log (click to expand)
# Time (UTC) Agent Action Result Files Tasks Progress Commit Gate
0 2026-04-26 14:51:16 Codex wait (missing-agent-label-transient) skipped 0 0/4
0 2026-04-26 14:51:51 Codex wait (missing-agent-label-transient) skipped 0 0/4 cancelled
0 2026-04-26 14:54:13 Codex wait (missing-agent-label-transient) skipped 0 0/4 failure
0 2026-04-26 15:00:20 Codex wait (missing-agent-label-transient) skipped 0 0/4 success
0 2026-04-26 15:03:13 Codex wait (missing-agent-label-transient) skipped 0 0/4 success

@stranske
stranske temporarily deployed to agent-high-privilege April 26, 2026 14:51 — with GitHub Actions Inactive
@stranske
stranske force-pushed the codex/verifier-codex-cli-0125 branch from 553fac4 to 173c8a8 Compare April 26, 2026 14:54
@stranske
stranske temporarily deployed to agent-high-privilege April 26, 2026 14:54 — with GitHub Actions Inactive
@stranske
stranske marked this pull request as ready for review April 26, 2026 14:58
Copilot AI review requested due to automatic review settings April 26, 2026 14:58
@stranske
stranske merged commit 72221b2 into main Apr 26, 2026
50 checks passed
@stranske
stranske deleted the codex/verifier-codex-cli-0125 branch April 26, 2026 14:58
@stranske
stranske temporarily deployed to agent-high-privilege April 26, 2026 14:59 — with GitHub Actions Inactive

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the pinned Codex CLI version used by the verifier reusable workflow, and aligns the workflow test to validate the new pin.

Changes:

  • Bump @openai/codex install pin in reusable-agents-verifier.yml to 0.125.0.
  • Update verifier workflow test to assert the new pinned version is installed.
  • Minor Python formatting cleanups in repo review evaluator code/tests.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/reusable-agents-verifier.yml Updates the Codex CLI version installed for verifier runs.
tests/workflows/test_verifier_terminal_disposition.py Adds an assertion ensuring the workflow installs the updated Codex CLI version.
scripts/repo_review_evaluator.py Minor formatting simplification of a list comprehension.
tests/scripts/test_repo_review_evaluator.py Minor formatting adjustment for readability/consistency.

Comment on lines 486 to 491
- name: Install Codex CLI
if: steps.context.outputs.should_run == 'true' && inputs.mode != 'evaluate'
run: |
set -euo pipefail
npm install -g "@openai/codex@0.101.0"
npm install -g "@openai/codex@0.125.0"
echo "Installed Codex CLI version: $(codex --version)" >> "$GITHUB_STEP_SUMMARY"

Copilot AI Apr 26, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This step uses npm install -g but the workflow doesn’t explicitly set up/pin a Node.js runtime. If you want this reusable workflow to be less sensitive to runner image changes (or to work more predictably on self-hosted runners), consider adding an actions/setup-node step with a pinned node-version before installing Codex.

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants